wayland: Don't traverse transient-ofs when faking root coordinate space

The position of each transient-of will be in fake-root coordinate
space; thus we should not accumulate all the positions making it an
offset; each window is already in fake root coordinate space.

https://bugzilla.gnome.org/show_bug.cgi?id=769402

wayland: Use effective toplevel as popup parent

When using the set transient-for as a popup parent, fetch the effective
toplevel instead, otherwise we will position against the wrong
coordinate.

https://bugzilla.gnome.org/show_bug.cgi?id=769402

Use gtk_popover_popdown/popup where appropriate

https://bugzilla.gnome.org/show_bug.cgi?id=769706

GtkPopover: Deprecate transitions-enabled

The effect of transitions-enabled=true can now be
achieved using gtk_popover_popup/popdown and the effect
of transitions-enabled=false can be achieved using
gtk_widget_show/hide.

https://bugzilla.gnome.org/show_bug.cgi?id=769706

GtkPopover: Add gtk_popover_popdown/popup

Since not chaining up in gtk_widget_show/gtk_widget_hide is not allowed,
we can't just implicitly delay the hiding in GtkPopover's hide
implementation. Fix this by introducing gtk_popover_popup() and
gtk_popover_popdown() to show or hide a popover with transition and
revert GtkPopover's show/hide implementation to apply their effect
without the transition.

https://bugzilla.gnome.org/show_bug.cgi?id=769706

Updated Indonesian translation

Updated Indonesian translation

mir: group DND, tooltips, and notifications with menu-type windows

Remove outdated comments

https://bugzilla.gnome.org/show_bug.cgi?id=768138

Use Menus to implement tooltips

The order in which tooltips are created, drawn, shown and then positioned,
always requires repositioning the surface. The tooltip window type only has
limited capability to do so. An alternative could be to use bufferstreams.

https://bugzilla.gnome.org/show_bug.cgi?id=768138

Fix execution of dialog

When a dialog is created, the mir event source is already executed on the
call stack. So without the recurse flag it will not be run in the main loop
used for the dialog.

https://bugzilla.gnome.org/show_bug.cgi?id=768138

file chooser: align date formatting with nautilus

We were treating 'yesterday' not quite the same way.

https://bugzilla.gnome.org/show_bug.cgi?id=769568

docs: Fix a dangling link

https://bugzilla.gnome.org/show_bug.cgi?id=769601

Really fix unlinkage for shm_open

Reported in https://bugzilla.gnome.org/show_bug.cgi?id=769603

Only apply type hint if it would map to a different mir surface type

Signed-off-by: Andreas Pokorny <andreas.pokorny@canonical.com>

Apply geometry hints to mir surface

Signed-off-by: Andreas Pokorny <andreas.pokorny@canonical.com>

Apply and forward title changes of gdk windows

Signed-off-by: Andreas Pokorny <andreas.pokorny@canonical.com>

Only update surface spec when there is no spec change pending

Rework window construction - only recreate surface when necessary

Signed-off-by: Andreas Pokorny <andreas.pokorny@canonical.com>

Forward repeated key presses as further down keys

Signed-off-by: Andreas Pokorny <andreas.pokorny@canonical.com>

Fix warning on newer version so mir 0.22 and newer

Signed-off-by: Andreas Pokorny <andreas.pokorny@canonical.com>

Fix gcc warning on potentially uninitialized gdk_event.

Use the surface output event to keep track of the suggested scale value

The event will be sent when the surface becomes visible on an output. With this change the GdkMirWindowImpl keeps track of the scale value and sends a configure event on change.

Signed-off-by: Andreas Pokorny <andreas.pokorny@canonical.com>

widget-factory: Don't initially show popover

It'll  be set as the popover of a GtkMenuButton later anyway which will
hide it again.

popover: Make some public API NULL-safe

Instad of potentially dereferencing a NULL-pointer, use the NULL-safe
gtk_popover_get_instance_private before the g_return_if_fail
precondition check.

label: Remove some useless casts

These were casts of a const GdkEvent* to GdkEvent* even though the
function they are passed to takes a const GdkEvent*.

Use g_snprintf instead of snprintf.

snprintf is not compatible with msvc.

git.mk: Add config.rpath to the list of ignored files

This finally cleans up the `git status` output.

Remove config.rpath from EXTRA_DIST

The config.rpath script was added by running gettextize, but it's not
really needed, as config.rpath is generated by autotools and it's
included in the dist tarball automatically.

Upstream gettextize has been fixed so it does not happen any more:
http://savannah.gnu.org/bugs/index.php?48729

Updated Czech translation

Updated Slovak translation

wayland: Link against librt

This is required for shm_open.

https://bugzilla.gnome.org/show_bug.cgi?id=769603

headerbar: Fix size allocation

The GtkHeaderBar gadget implementation was subtly broken: it called
gtk_widget_set_allocation both in gtk_header_bar_size_allocate (with
the actual allocation) and in gtk_header_bar_allocate_contents (with
the content allocation of the main gadget). Dropping the second call
fixes the render node conversion for GtkHeaderBar.

Updated Hebrew translation

Updated Hebrew translation

gdkwindow: Fix potential NULL pointer dereferences in event code

The event code could potentially dereference pointer_info if the
invariant that ENTER_NOTIFY and LEAVE_NOTIFY events are only emitted on
devices which have pointers is violated elsewhere.

Found with scan-build.

https://bugzilla.gnome.org/show_bug.cgi?id=712760

gtkicontheme: Don’t emit warning about fallback theme when in unit tests

In order to eliminate g_test_expect_message() (which doesn’t work with
G_LOG_USE_STRUCTURED), make the warning about the fallback theme not
existing be conditional on the icon theme search path containing a
system path. Any application code which modifies the search path does so
through appends and prepends, so this should not affect whether the
warning is emitted in production.

https://bugzilla.gnome.org/show_bug.cgi?id=769485

tests: Avoid g_test_expect_message

Instead of using g_test_expect_message, just ignore the warning.

GtkMenuButton: use :toggled instead of :clicked

:toggled is triggered on :clicked, so using :toggled lead to the menu
to be popped up at the same time, while allowing to use the toggle state
and avoiding any need to a hack to prevent recursion, which somehow
wasn't enough for double emission of GtkMenuToolButton:show-popup.

https://bugzilla.gnome.org/show_bug.cgi?id=769287

Updated Polish translation

csslookup: Remove useless cast

The style passed to _gtk_css_lookup_resolve is already a
GtkCssStaticStyle.

Updated German translation
(cherry picked from commit 070c4f74868c31cc9316bbad38e2ad42adc43443)

W32: Prefer the deadkey combinations that the OS uses

Pick the W32 API for possible deadkey+<something> combinations
and prefer these to other sources of deadkey combos.
Specifically, if W32 API supports at least one combo for a particular
deadkey, only use that data and do not attempt to do other, unsupported
combinations, even if they make sense otherwise.
This is needed to, for example, correctly support US-International
keyboard layout, which produces a combined character for <' + a>
combo, but not for <' + s>, for example.

This is achieved by stashing all the deadkeys that we find in
an array, then doing extra loop through all virtual key codes and
trying to combine them with each of these deadkeys. Any combinations
that produce a single character are cached for later use.

In GTK Simple IM context, call a new GDK W32 function to do a lookup
on that cached combination table early on, among the "special cases"
(which are now partially obsolete).

A limitation of this code is that combinations with more than
one deadkey are not supported, except for combinations that consist
entirely of 2 known deadkeys. The upshot is that lookups should
be relatively fast, as deadkey array stays small and the combination
tree stays shallow.

Note that the use of ToUnicodeEx() seems suboptimal, as it should
be possible to just load a keyboard library (KBD*.DLL) manually
and obtain and use its key table directly. However, that is much more
complicated and would result in a significant rewrite of gdkkeys-win32.
The code from this commit, though hacky, is a direct addition to
existing code and should cover vast majority of the use-cases.

https://bugzilla.gnome.org/show_bug.cgi?id=569581

GDK W32: Cache multiple keyboard layouts simultaneously

This changes the group/level semantic.
Previously W32 backend used "group 0/1" to denote "AltGr OFF/ON"
and "level 0/1" to denote "Shift is OFF/ON".
Now "group" means "keyboard layout" and there can be up to 255 groups,
while AltGr and Shift are combined into a single level enum that
takes values between 0 and 4.
Unlike X, W32 doesn't do effective group overriding, meaning that
it will never tell the caller that a different group was actually
used (even for universal keys, such as Enter), because key symbol
table is completely fabricated and there's no point in trying to
save a few of kilobytes of RAM by not duplicating universal key
records for all groups.

Also contains many whitespace changes (tab elimination, fixed
indentation) and cleanup (axed a few global variables, these are
now accessed via the default keymap).

https://bugzilla.gnome.org/show_bug.cgi?id=768722

shortcutlabel: Make parameter names match

widget: remove some unneeded function prototypes

filechooserwidget: Remove unused enum

label: Destroy the popup menu on unmap

Updated Spanish translation

Modify git.mk to handle our split NLS directories

GTK+ has two directories for translations: the default 'po' and the
additional 'po-properties' for the GObject properties translation
domain.

Since the content of the translations directories are filled by
autoreconf and gettext, and are duplicated between the two gettext
domains we use, we should handle this ad hoc inside git.mk, instead of
trying to catch up by adding po-properties files in GITIGNOREFILES.

Add ignore file for macro dir

The contents of the macro directory are too complicated for git.mk to
handle: the contents are filled by autotools and may change between
systems or autotools releases; and we additionally ship our own m4
macros.

To avoid unnecessary noise on the `git status` output we should simply
add an ignore file specifically for the macro directory.

Skip GtkShortcutLabel::accelerator in notify test

This property is not freely settable, since the string
gets parsed.

menu: Fix up new properties

The new positioning-related properties had some quality of
implementation issues, such as incorrect initial values and
excessive change notification. This broke the notify test.

Make icontheme test work with structured logging

Since we've opted in to G_LOG_USE_STRUCTURED, this test
now needs to do its thing using a log writer function.

Skip GdkDrawingContext in default value tests

GdkDrawingContext objects can't be freely constructed.

Require a new version of GLib

We need GLib 2.49.4 because of the new API requirements introduced by
commit 936c1649.

https://bugzilla.gnome.org/show_bug.cgi?id=769451

Updated Spanish translation

Updated Spanish translation

Update the docs for gtk_window_get_position()

Drop mentions of GnomeClient, and add mentions of different windowing
systems instead of just assuming that we're using X11.

Add a release not about GtkTable expand flags

gtkbox: remove unnecessary queue_compute_expand()

The expand child property does not have any effect on the
expand state of the GtkBox, so queuing a compute_expand
when changing it is not needed.

https://bugzilla.gnome.org/show_bug.cgi?id=769162

gtktable: don't try to propagate expand related child props in compute_expand()

It tried to set the expand state if either xexpand/yexpand where true.
Due to a missing queue_compute_expand when adding a child it actually
only computed the expand state in case a child queued after being added
or in case a child had the expand property set (see optimization in
gtk_widget_set_parent)

In my case this broke layouts as a child of GtkCombBox started setting
an exand flag with 3.20 which queued a compute_expand, which in turn
propagated an expand child props set for a cell in the same table up
and overrode the expand child prop of a parent GtkBox.

This removes the custom compute_expand implementation to match the
behaviour of GtkBox (don't propagate child prop expand flags
but let child expand flags override the child props) and not get random
expand behaviour depending on whether and when child widgets set their
expand state.

https://bugzilla.gnome.org/show_bug.cgi?id=769162

Updated Lithuanian translation

demos: Fix build failure

gtk+/demos/gtk-demo/css_blendmodes.c: In function ‘update_css_for_blend_mode’:
gtk+/demos/gtk-demo/css_blendmodes.c:49:26: error: format not a string literal, argument types not checked [-Werror=format-nonliteral]
                          blend_mode);
                          ^~~~~~~~~~

https://bugzilla.gnome.org/show_bug.cgi?id=769236

docs: trivial build fix

The error was:
gtk3-scan.c:193: undefined reference to `gtk_shortcuts_label_get_type'
collect2: error: ld returned 1 exit status

It's since commit 7543cd8ce419a4d660f0ff7614f0c6e1cac81804, which made
the GtkShortcutLabel class public.

Port GtkPrintOperationPortal to gtk_window_export_handle

Port gtk_show_uri_on_window to gtk_window_export_handle

As part of this, make it use the new
g_app_info_launch_default_for_uri_async, since we need a way
to unexport the window handle afterwards.

Port GtkFileChooserNativePortal to gtk_window_export_handle

Put window exporting behind a display protocol agnostic API

Introduce a private API meant for abstracting how to get a handle
of a window that can be shared with other processes. The API is
async, since some implementations will require that. Currently,
only X11 is supported, which doesn't.

Based on a patch by Jonas Adahl.

GDK W32: Support UTF-16 surrogate pairs passed via VK_PACKET

This is, essentially, a piece of g_utf16_to_ucs4() built into GDK
W32 keyboard message processing.

https://bugzilla.gnome.org/show_bug.cgi?id=769126

css: Fix case where we didn't convert pt => px

Fixes gtk_widget_override_font() not behaving properly since commit
df08fc91bdc1d2e4c866122304fabe4dd298a7de.

https://bugzilla.gnome.org/show_bug.cgi?id=768902

css: Rename functions

I don't want to use a generic function for font sizes as font sizes are
special. Plus, the function is only used in one place.

Properly declare the font CSS property

https://bugzilla.gnome.org/show_bug.cgi?id=769004

Don't use one-stop gradients

They are deprecated, which causes a warning that makes
the test fail.

https://bugzilla.gnome.org/show_bug.cgi?id=769004

Fix tiny typo.

Adwaita: use a transition to animate checks and radios

see https://bugzilla.gnome.org/show_bug.cgi?id=762260

shortcut-label: add 'disabled-text' property

When there's no useful shortcut accelerator set,
GtkShortcutLabel doesn't show any useful information.

To work around that, add a new property to set the
text to be displayed when there's no accelerator
available.

https://bugzilla.gnome.org/show_bug.cgi?id=769205

shortcut-label: make it public

GtkShortcutLabel is a widget that displays a single
shortcut accelerator or gesture in the user interface,
and is currently used by the shortcuts window.

This widget, however, has public value as other applications
also may want to expose their own shortcuts. For instance,
it'll be useful for the Keyboard panel on Control Center and
the new shortcut editor in Pitivi, among others.

This patch exposes GtkShortcutLabel as a public widget,
and adds the necessary documentation.

https://bugzilla.gnome.org/show_bug.cgi?id=769205

GtkScrolledWindow: reset scroll history if it contained no real history

Scroll history must refer to a timespan for the values to be valid, otherwise
we return FALSE, in this case the stored event(s) should be discarded anyway.

GtkScrolledWindow: Always uninstall scroll cursor when starting deceleration

It could be the case that the last scroll event is received long after any
previous scroll event, in this case the last scroll event discards all "old"
scroll events, and scroll_history_finish() returns FALSE because there's no
time/offset deltas in the scroll history.

This is desired so we don't trigger the deceleration effect if there was no
effective velocity, we still must reset the installed scroll cursor, so take
it out of this if() condition.

Updated Portuguese translation

Stop using one-stop gradients in reftests

This is causing warnings now that make the tests fail.

Revert a mistaken change

I thought I needed ot rearrange the ordering of the animation-direction
values for the parser, overlooking the fact that we already parse them
backwards to address this very problem.

docs: GtkWidget::style-updated vs GtkStyleContext::changed

Explain the difference between those two signals.

Add "Since: 3.0" for GtkStyleContext::changed, since that signal has
been added in commit 9f84e101bf150a85f82e15c56130b1f98b8fdbf0, present
since 2.91.6.

https://bugzilla.gnome.org/show_bug.cgi?id=769047

docs: add missing info for gtk_widget_get_style_context()

It is important to know whether the returned object can or cannot
change, for a certain widget. For example to connect to the
GtkStyleContext::changed signal.

https://bugzilla.gnome.org/show_bug.cgi?id=769047

GtkApplication: some cleanups

Always return an error if we fail to get a dbus proxy; the callers
are only looking whether error is set, not whether the return value
is NULL.

Use the same function for the inhibit proxy as well, and clean up
the sm_proxy in finalize.

Don't apply GDK_HINT_RESIZE_INC to GDK_WINDOW_STATE_TILED windows

This matches the behaviour of Mutter, Metacity and traditional X11
window managers on the window manager side, and is what we want
for at least gnome-terminal. I can't think of any reason why we'd
want incremental resize in any other tiled window.

Signed-off-by: Simon McVittie <smcv@debian.org>
Bug: https://bugzilla.gnome.org/show_bug.cgi?id=760944

https://bugzilla.gnome.org/show_bug.cgi?id=755947

reftests: override GSETTINGS_SCHEMA_DIR when running tests

The reftest depends on a schema we install ourselves. Instruct GIO to
look in the build directory to avoid a cyclic dep.

https://bugzilla.gnome.org/show_bug.cgi?id=768930

Avoid a division by zero

It appears that xvfb returns nonsense in its xrandr resources.
Avoid a crash when that happens.

https://bugzilla.gnome.org/show_bug.cgi?id=768999

Use g_clear_object in a few more places

Adwaita: use margins for entry image spacing

there used to be a padding, since it was the only option in the
past, that makes little sense now, hence use margins.

HC: export unfocused_insensitive_color...

...to reflect Adwaita changes.

Adwaita: export yet another named color for insensitive bg/fg...

...which was needed, but not there, use responsibly.

Fix a few documentation parameter names

Opt in to structured logging

Define G_LOG_USE_STRUCTURED, so that all our g_debug, g_warning,
etc calls directly use structured logging and provide source information.

wayland: Fix build

Missing include pointing to tablet-unstable-v1-client-protocol.h,
pays me for not testing on a clean checkout.

wayland: Use wl_fixed_t on wp_tablet_tool angle arguments

This is an incompatible change in tablet protocol v2.

wayland: Update current tablet support to using v2

Only update to using v2 headers/structs. The incompatible changes
to tool events are dealt with in the next commit. Pads aren't handled
in this commit either.

Adwaita: dim calendar `uninteresting` days some more

see https://bugzilla.gnome.org/show_bug.cgi?id=769003

Adwaita: clean calendar styling

remove some non working props and generally simplify and clean up
the code, in the process address:

https://bugzilla.gnome.org/show_bug.cgi?id=769003